home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part2 / 10688 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  1.6 KB
  1. Path: ix.netcom.com!news
  2. From: miker3@ix.netcom.com (Mike Rubenstein)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: Converting Strings to Upper Case
  5. Date: Tue, 19 Mar 1996 13:43:03 GMT
  6. Organization: Netcom
  7. Message-ID: <314eb83f.308884702@nntp.ix.netcom.com>
  8. References: <4ifra6$52i@scipio.cyberstore.ca> <4ih7l3$526@thrush.sover.net>  <4ihl4m$4ca@castle.nando.net> <NEWTNews.827184423.14391.breese@PC_Breese.physio-control>
  9. NNTP-Posting-Host: ix-dc8-01.ix.netcom.com
  10. X-NETCOM-Date: Tue Mar 19  7:42:01 AM CST 1996
  11. X-Newsreader: Forte Agent .99d/32.182
  12.  
  13. Brian Reese <breese@kahuna> wrote:
  14.  
  15. > 
  16. > Here's my offering:
  17. > 
  18. > #include <stdio.h>  /* for printf()  */
  19. > #include <ctype.h>  /* for toupper() */
  20. > 
  21. > char * strtoupper( char * );  /* converts string to upper case */
  22. > 
  23. > int main( void )
  24. > {
  25. >     char * test = "my test string";
  26. >     
  27. >     printf( "%s\n", test );
  28. >     printf( "%s\n", strtoupper( test ) );
  29. >     
  30. >     return 0;
  31. > }
  32. > 
  33. > char * strtoupper( char * ptr )  /* converts string to upper case */
  34. > {
  35. >     char * save_ptr = ptr;   /* save pointer for return */
  36. >     
  37. >     while ( *ptr = toupper( *ptr++ ) )
  38. >         ;
  39. >     return save_ptr;
  40. > }
  41.  
  42. Not a very good offering.  There are two obvious problems in your
  43. example.
  44.  
  45. First, the identifier strtoupper with external linkage is reserved.
  46. Defining your own function with external linkage with this name
  47. results in undefined behavior.
  48.  
  49. Second, the expression
  50.  
  51.     *ptr = toupper( *ptr++ )
  52.  
  53. modifies ptr and also acesses its value other than to determine the
  54. new value without an intervening sequence point.  This too results in
  55. undefined behavior.
  56.  
  57. Michael M Rubenstein
  58.